Skip to content

Conversation

@sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Jul 22, 2025

Regular tox update

Includes a small openai-agents tests fix and restricts openai-agents tests to 3.10 as in 0.2.x they're using 3.10+ typing syntax (<Type> | <Type>)

@codecov
Copy link

codecov bot commented Jul 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.63%. Comparing base (e329179) to head (731cba4).
Report is 1 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4609      +/-   ##
==========================================
- Coverage   80.69%   80.63%   -0.07%     
==========================================
  Files         156      156              
  Lines       16500    16500              
  Branches     2806     2806              
==========================================
- Hits        13314    13304      -10     
- Misses       2297     2305       +8     
- Partials      889      891       +2     

see 5 files with indirect coverage changes

@sentrivana sentrivana marked this pull request as ready for review July 23, 2025 08:08
@sentrivana sentrivana requested a review from a team as a code owner July 23, 2025 08:08
"deps": {
"*": ["pytest-asyncio"],
},
"python": ">=3.10",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not 3.9? (the lib supports it according to pypi)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See PR description

(...) restricts openai-agents tests to 3.10 as in 0.2.x they're using 3.10+ typing syntax ( | )

@sentrivana sentrivana merged commit fe3ccb8 into master Jul 24, 2025
137 checks passed
@sentrivana sentrivana deleted the ivana/tox/new-stuff branch July 24, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants